Skip to content
This repository was archived by the owner on Oct 21, 2024. It is now read-only.

FIX: Correct handling of unused 'subset' parameter in fasttext_filter (#54) #55

Open
41ow1ives wants to merge 1 commit into
base: main
Choose a base branch
from
Open

FIX: Correct handling of unused 'subset' parameter in fasttext_filter (#54) #55

41ow1ives wants to merge 1 commit into from

Conversation

@41ow1ives
Copy link
Collaborator

@41ow1ives 41ow1ives commented May 24, 2024 •
edited
Loading

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows dataverse guidelines link:
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What does this PR do?

@41ow1ives 41ow1ives added the Bug Something isn't working label May 24, 2024
@41ow1ives 41ow1ives requested review from eddie-liner and p-idx May 24, 2024 05:21
@41ow1ives 41ow1ives self-assigned this May 24, 2024
@p-idx
Copy link
Contributor

p-idx commented Jun 27, 2024

lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@eddie-liner eddie-liner Awaiting requested review from eddie-liner

@p-idx p-idx Awaiting requested review from p-idx

At least 1 approving review is required to merge this pull request.

Assignees

@41ow1ives 41ow1ives

Labels

Bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@41ow1ives @p-idx